Skip to content

fix last order column only when DML #516

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Oct 14, 2023

Conversation

sean-k1
Copy link
Collaborator

@sean-k1 sean-k1 commented Oct 13, 2023

resolve #510

Description

Over pymyrepli 1.0 version
We got column Information from optional metadata

BUT If Mysql 5.7 Users use over 1.0 version They do not get Column Information.
Mysql 5.7 version does not have binlog_row_metadata variable. so we support 0.45.1 version for mysql 5.7 version

Nevertheless If Mysql 5.7 version use over 1.0 Version, users found only last order column Values.

When we do not know column Information values dictionary key is always null, So overwrite value Object.

values[name] = self.__read_values_name(
          column,
          null_bitmap,
          null_bitmap_index,
          cols_bitmap,
          unsigned,
          i,
      )

Solution

To avoid user confusion, it is marked and displayed as UNKNOWN_COL.

When we do not know columnInformation
values dictionary key is always null
so overwrite value Object.
Thus always values dictionary has last order column object value

To avoid user confusion, it is marked and displayed as UNKNOWN_COL.
@sean-k1 sean-k1 requested a review from dongwook-chan October 13, 2023 07:10
@sean-k1 sean-k1 self-assigned this Oct 14, 2023
@sean-k1 sean-k1 merged commit 72b53a8 into julien-duponchelle:main Oct 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Untrusted binlog row snapshot data >=v1.0
1 participant